-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(neon_framework): rename credentials password to appPassowrd #2488
refactor(neon_framework): rename credentials password to appPassowrd #2488
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
loginName and username can not be mixed and we need both.
packages/neon_framework/packages/account_repository/lib/src/account_repository.dart
Outdated
Show resolved
Hide resolved
packages/neon_framework/packages/talk_app/lib/src/blocs/room.dart
Outdated
Show resolved
Hide resolved
Signed-off-by: Nikolas Rimikis <[email protected]>
…dentials Signed-off-by: Nikolas Rimikis <[email protected]>
ff5ce04
to
5addca0
Compare
I just dropped the login name change and will fix that in the UserDetails bloc refactor. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2488 +/- ##
==========================================
- Coverage 30.29% 30.26% -0.03%
==========================================
Files 352 352
Lines 124389 124320 -69
==========================================
- Hits 37680 37631 -49
+ Misses 86709 86689 -20
*This pull request uses carry forward flags. Click here to find out more.
|
As discussed in the initial account_repository PR (#2403 (comment)).